Skip to content

Replace swagger urls in org.springdoc.core.properties.AbstractSwaggerUiConfigProperties#urls only if url is changed. Test to check if the issue is fixed. Fixes #2509. #2668

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

dgswan
Copy link
Contributor

@dgswan dgswan commented Aug 4, 2024

Replace swagger urls in org.springdoc.core.properties.AbstractSwaggerUiConfigProperties#urls only if url is changed. Test to check if the issue is fixed. Fixes #2509.

…rUiConfigProperties.urls` only if url is changed. Test to check if the issue is fixed. Fixes springdoc#2509.
@dgswan dgswan force-pushed the bugfix/T-2509-fix-array-index-out-of-bounds-exception-in-swagger-ui-config-parameters branch from 732a2e8 to 2af8626 Compare August 4, 2024 13:45
…figProperties.urls` and replace it with mutation of `org.springdoc.core.properties.AbstractSwaggerUiConfigProperties.SwaggerUrl.url` when necessary. Enhance `SpringDocApp8MultipleUrlsSeveralParallelRequestsTest` by specifying custom servlet path to check no exception is thrown.
@bnasslahsen bnasslahsen merged commit 1d61661 into springdoc:main Sep 17, 2024
1 check passed
@dgswan dgswan deleted the bugfix/T-2509-fix-array-index-out-of-bounds-exception-in-swagger-ui-config-parameters branch September 18, 2024 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ArrayIndexOutOfBoundsException in SwaggerUiConfigParameters
3 participants